Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

hwdef: Add support for Sierra-F1 #24695

Closed
wants to merge 2 commits into from

Conversation

MallikarjunSE
Copy link
Contributor

Hey @tridge @bugobliterator
Possible to check this? There are few changes suggest by Sid for ADC3 support.
Thank you!

@MallikarjunSE
Copy link
Contributor Author

Hey @tridge @bugobliterator
I have used define HAL_USE_ADC FALSE since ADC3 is not working. I want to go ahead with this for now.
Thank you!

# as an alternative config setup the RX6 pin as a uart. This allows
# for bi-directional UART based receiver protocols such as FPort
# without any extra hardware
PC7 USART6_RX USART6 ALT(1)
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

might need to set its default protocol to RC input in order to assure a DMA is assigned... @tridge would know for sure

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

if you intend this to also communicatee with an IOMCU for more PWM, I think you may need to add something to the hwdef for the coprocessor use.... @tridge would be able to comment

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

if you intend this to also communicatee with an IOMCU for more PWM, I think you may need to add something to the hwdef for the coprocessor use.... @tridge would be able to comment

It's planned for future use. I'm planning to add STM32L431xx for more expansion. I want to get this base setup running to plan the future ones.

</table>


### GPS + RCIN(UART6)
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this is not correct, right? no I2C for compass, UART6 RX is not on the connector

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hey Henry, just checked. We have I2C only on power ports. This connector has UART6 or RCIN.

Copy link
Collaborator

@Hwurzburg Hwurzburg Aug 20, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

that's unfortunate....so external GPS/compass modules (the typical kind) cant connect to the GPS connector....so only CAN GPS/Compass can be used (using an internal compass is usually too prone to motor interference)....
in any case the GPS/RC connector show TX1/TX2 and RX1/RX2 not UART6 and UART7(GPS)

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes. Most of our customers use CAN RM3100 for magnetometer. And I2C is there on Power ports for smart battery SMBus few our customers use.

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

again, GPS is TX7/RX7 and RC is RX6....that is not what is listed below

@MallikarjunSE MallikarjunSE force-pushed the pr-sierra-f1 branch 2 times, most recently from db3f54e to a1c436a Compare August 21, 2023 07:51
@MallikarjunSE MallikarjunSE force-pushed the pr-sierra-f1 branch 2 times, most recently from f0d62b4 to 3523e7b Compare September 3, 2023 12:00
@Hwurzburg Hwurzburg added the WikiNeeded needs wiki update label Sep 3, 2023
PC15 EXT_LDO_EN OUTPUT HIGH

# PWM channels
PE9 TIM1_CH1 TIM1 PWM(1) GPIO(50)
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

should try to make these bi-directional if possible

- SERIAL0 -> USB
- SERIAL1 -> UART2 (Telem1, DMA-enabled)
- SERIAL2 -> UART8 (Telem2, DMA-enabled)
- SERIAL3 -> UART7 (GPS, NODMA)
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this is not the order you defined them in in the hwdef!

</table>


### GPS + RCIN(UART6)
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

again, GPS is TX7/RX7 and RC is RX6....that is not what is listed below

</tr>
<tr>
<td>3 (blk)</td>
<td>RX (IN)</td>
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

RX2

</tr>
<tr>
<td>2 (blk)</td>
<td>TX (OUT)</td>
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

TX2

</tr>
<tr>
<td>3 (blk)</td>
<td>SCL I2C</td>
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

should note which I2C bus....3 or 4 on the SCL and SDA pins

# LED drivers
NTF_LED_TYPES 289

# EKF setup
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

these are not required.....already defaults

@@ -0,0 +1,13 @@
# Serial3 default to GPS
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

my mistake....SERIAL3 already defaults to GPS....so not needed.....but need to fix UART order in hwdef so that UART7 is third and UART6 is forth, not the other way around...

@tridge
Copy link
Contributor

tridge commented Feb 5, 2024

@MallikarjunSE when I enable ethernet it watchdogs, we'll need to debug that

PA7 ETH_RMII_CRS_DV ETH1
PA1 ETH_RMII_REF_CLK ETH1

define BOARD_PHY_ID MII_LAN8742A_ID
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

is it a LAN8742A ?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes.

@tridge
Copy link
Contributor

tridge commented Sep 4, 2024

@MallikarjunSE can you send me a photo of exactly what the device you sent me looks like - I'm embarrassed to say I can't find it! @peterbarker found this PR in his archaeology efforts

@tridge tridge removed the DevCallEU label Sep 4, 2024
@tridge
Copy link
Contributor

tridge commented Sep 4, 2024

I've found this board and started debugging ethernet support
The failure is that it is waiting forever (until we watchdog) for the ETH_DMAMR SWR bit to clear.
image

@tridge
Copy link
Contributor

tridge commented Sep 4, 2024

@MallikarjunSE I suspect the PHY is not properly connected to the H743. Please send me the schematic and also double check the layout and connections

@Hwurzburg
Copy link
Collaborator

@MallikarjunSE status....if this is not active, we should close the PR

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants